-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Moving service worker job related code to its own file. #628
Moving service worker job related code to its own file. #628
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice first step! This looks good to me. Thanks for doing the refactor job. Would love @jellefoks to take a look as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks for taking the first stab at this. I hope that maybe sometime later we won't need a 'friend class' between these two either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make sure the /internal directory/submodule doesn't get in, git submodules cause a lot of havoc.
Other than that looks like a great refactor
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
c3d2d56
to
ab2a2d4
Compare
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61 (cherry picked from commit 6e280c8)
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61 (cherry picked from commit 6e280c8)
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61 (cherry picked from commit 6e280c8)
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61 (cherry picked from commit 6e280c8)
b/286597506 Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61 (cherry picked from commit 6e280c8) Co-authored-by: aee <[email protected]>
b/286597506
Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61